Skip to content

Update Modbus TCP register version to 3.60#384

Closed
iRateD1703 wants to merge 4 commits intosfstar:mainfrom
iRateD1703:patch-1
Closed

Update Modbus TCP register version to 3.60#384
iRateD1703 wants to merge 4 commits intosfstar:mainfrom
iRateD1703:patch-1

Conversation

@iRateD1703
Copy link

@iRateD1703 iRateD1703 commented Oct 20, 2025

Breaking change

Proposed change

New version of Modbus TCP register version 3.60 which introduces new datatypes for previously overflowing variables. The const file is updated accordingly.

Type of change

  • Dependency upgrade
  • Bugfix (non-breaking change which fixes an issue)
  • New integration (thank you!)
  • New feature (which adds functionality to an existing integration)
  • Deprecation (breaking change to happen in the future)
  • Breaking change (fix/feature causing existing functionality to break)
  • Code quality improvements to existing code or addition of tests

Additional information

Checklist

  • The code change is tested and works locally.
  • There is no commented out code in this PR.
  • The code has been formatted using Ruff (ruff format custom_components/victron)

@iRateD1703
Update const.py
dd9f507 
Updated data types for solarcharger_yield_power and solarcharger_yield_user to UINT32 with MOdbus TCP register list 3.60
@iRateD1703
Updated to v 3.60
e93dc76 
Updated to Modbus TCP register v 3.60 which includes appropriate datatypes for user yield and user power
@iRateD1703
Updated to v 3.60
ee7dc51 
Updated to Modbus TCP register version 3.60 which now includes appropriate datetypes for previously overflowing unit16 variables
@iRateD1703
Updated to TCP register version 3.60
d3e1110 
Updated to Modbus TCP register version 3.60 which now includes appropriate datetypes for previously overflowing variables
@sfstar
Copy link
Owner

sfstar commented Nov 2, 2025

Thank you for submitting a pr.
I went for merging #386 my stale branch in which registers up to 3.66 were included (This is now part of the beta-3 pre-release for this integration).
On that branch I went for a slightly different approach (adding new uint32/int32 registers as new entities as not to break older installation).
Feel free to check it the pre-release

@asmcc asmcc mentioned this pull request Nov 21, 2025
Closed
@github-actions
Copy link

github-actions bot commented Jan 2, 2026

This pull request is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days

@github-actions github-actions bot added the stale There has not been activity on this issue or PR for quite some time. label Jan 2, 2026
@github-actions github-actions bot closed this Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

stale There has not been activity on this issue or PR for quite some time.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Wrong yield total amount

2 participants

@iRateD1703 @sfstar